Repository navigation
Release v0.5.1 - #277
Merged
Merged
Release v0.5.1#277
Conversation
…k for the 7-step app
- pyproject, the notebook's current_version/RELEASE_TAG, the version index and the README
pins: 0.5.1. CHANGELOG "Unreleased" becomes "v0.5.1 (10 November 2026)" with a user-facing
summary and a migration note for users and bricks.
- Notebook: cell 3 calls PreprocessingNotebook(); every code cell is a Colab form
("cellView": "form"); a cell that cannot go on raises its own CellStop, shown as one line
without a traceback; cell 3 on Colab without Drive opens the app after one line; cell 1
checks the Python version before importing tomllib; the introduction describes the
seven-step app.
- Tests: the one-line stops in a real kernel (cell 3 alone, a refused checkout), faked Colab
without Drive, the cellView metadata, the version check before tomllib.
- docs/colab-release-check.md rewritten for the three cells and the seven steps.
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01G3qp9AxjpGPw9xfWAUJNqc
RELEASE_REF = d023ff9 (and the tests asserting it), with its PreprocessingNotebook() call recorded in RELEASE_CALLS, the same two-commit pattern as v0.5.0 (3b616a0, c74d717). A commit cannot hold its own SHA, so the notebook installs the preparation commit, whose package code is the release's. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01G3qp9AxjpGPw9xfWAUJNqc
check.py 5270 passed / 4 environment skips (listed as not run); Spec modules 1732 passed, 0 skipped, all 24 expected modules; install from RELEASE_REF d023ff9 verified. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01G3qp9AxjpGPw9xfWAUJNqc
The one-line CellStop of cell 1 and cell 3 for an interface that cannot load, and of cell 2
for a failed Drive mount, dropped the error that caused it: the chained ImportError (a missing
dependency, a broken import inside the package) or the mount error (for example "Mountpoint
must not already contain files") was not shown anywhere, and the advice could be wrong.
The stop now names the cause on the same line ("... could not load (ImportError: ...)",
"Google Drive could not be connected (ValueError: ...)").
Tests: the interface stop of cells 1 and 3 names its ImportError (exec), the Drive stop names
the mount error, and in a real kernel the interface stop is one line naming the
ModuleNotFoundError while an unexpected error in cell 3 keeps its full traceback.
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01G3qp9AxjpGPw9xfWAUJNqc
…bricks The bricks table named read_confirmed_design and RunInputs.design() but not the rest of the PP-02 removal: design.ExperimentDesign, DesignChanges, default_design, load_design, read_experiment_design, the modules design_edit and cli_design, run.design_file and report.design_summary/design_table (each checked: no longer importable at d023ff9). Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01G3qp9AxjpGPw9xfWAUJNqc
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01G3qp9AxjpGPw9xfWAUJNqc
Contributor
Author
Independent review: ready after 2 small fixesVerified
Fixed
No A/B redo was needed: nothing that Generated by Claude Code |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
The branch starts from
mainat9a5da03, after #256-#276 were merged:d023ff9Prepare v0.5.1 (commit A).0.5.1inpyproject.toml, in the notebook'scurrent_versionandRELEASE_TAG, innotebooks/notebook_latest_versions.yaml, in the README pins (@v0.5.1) and in the tests that pin it.docs/colab-release-check.mdis rewritten for the three cells and the seven steps. AGENTS.md, the user guide and the README describe how the cells now behave.a4154adPoints the notebook at commit A (commit B). SetsRELEASE_REF = "d023ff91289a2d2204e1ae56f7dee114d1a665e4", adds itsPreprocessingNotebook()pair toRELEASE_CALLSand updates the test that asserts it. This is the same two-commit pattern as v0.5.0 (3b616a0,c74d717).git diff d023ff9 a4154ad -- src pyproject.tomlis empty.7607617Release record indocs/conformance.md, measured ona4154ad. This commit changes only that file.Review fixes (details in the review comment):
7beac12: when a cell stops with one line, that line now names the error that caused the stop (notebook, tests and docs).9f61cb9: the CHANGELOG table for bricks now lists the removed design editor API.849e09d: the release record is measured again on9f61cb9: 5274 passed, 4 environment skips; Spec modules 1732 passed, 0 skipped.git diff d023ff9 849e09d -- src pyproject.tomlis empty, soRELEASE_REFstays A.The shipped notebook
PreprocessingNotebook(), withoutdescribe=. The call is still recorded in one place:test_notebook_file.RELEASE_CALLSmapsRELEASE_REFto it, with the new paird023ff9…→PreprocessingNotebook()(release checklist)."cellView": "form"in its metadata (AGENTS.md §7), and a test checks it.CellStop(RuntimeError), whose_render_traceback_()returns[message]. IPython shows that list in place of the traceback, so Colab and JupyterLab do the same.nbclient): the error output hastraceback == [evalue].exec, plus a real-kernel test withgoogle.colabfaked.import tomllibhas moved from the top of cell 1 into_checkout_version. The version check now comes right after the three stdlib imports and theCellStopclass, and an AST test checks this.Gate
python tools/check.pyon9f61cb9a4154ad: 5270 passed, 4 skipped (reproduced by the review)pytest -rsxX tests/test_spec_*.pyEXPECTED_MODULES. JUnit has no skipped, failure or error casepip install "morphobricks-preprocessing[notebook] @ git+…@d023ff9…"gives version 0.5.1 with normalization 0.4.0. The installed package files equal this branch'ssrc(diff -r).PreprocessingNotebook()buildswelcome(),steps()andresults_step()(threeVBoxes)RELEASE_REFfrom GitHub and printsReady: preprocessing 0.5.1; normalization 0.4.0; ipywidgets 8.1.9. Cell 2 prints its one-line note, and cell 3 displays the app. Fakedgoogle.colab: cell 1 also installs thezarrextra. Cell 3 run before cell 2 prints the Drive line and opens the app. After cell 2 mounts Drive, cell 3 opens without the lineNot run (environment skips, outside the Spec modules):
test_cli_log.py:166,test_folder_rules.py:199andtest_folder_rules.py:230: the file system is case-sensitive.test_discovery.py:665: the tests run as root.For the owner
docs/colab-release-check.md. Both links are pinned to the final head849e09d, which holds the reviewed notebook; it installsRELEASE_REFA:''https://colab.research.google.com/github/CellMigrationLab/morphobricks-preprocessing/blob/849e09d62b95341b285c5e2f1f2d725c5d7bbc5e/notebooks/MorphoBricks_Preprocessing.ipynb'''''https://raw.githubusercontent.com/CellMigrationLab/morphobricks-preprocessing/849e09d62b95341b285c5e2f1f2d725c5d7bbc5e/notebooks/MorphoBricks_Preprocessing.ipynb'''RELEASE_REFd023ff9stays reachable frommain.https://github.com/CellMigrationLab/morphobricks-preprocessing/releases/new?tag=v0.5.1&target=<merge sha>&title=v0.5.1@v0.5.1pins cannot be installed until the tag exists. The README Colab badge still links toblob/main.🤖 Generated with Claude Code
https://claude.ai/code/session_01G3qp9AxjpGPw9xfWAUJNqc